The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		Apr 30, 2026 0:30am

PR Summary

Medium Risk
Touches Monaco content synchronization during streaming, which can subtly affect cursor/scroll behavior and correctness of incremental updates, but is limited in scope and not security-sensitive.

Overview
Reduces flicker and preserves cursor/scroll position when syncing streamed file generation output into the Monaco editor by replacing model.setValue() with model.applyEdits().

When incoming content is purely additive, the editor now inserts only the new suffix at the end; otherwise it falls back to replacing the full model range, and removes the previous view-state save/restore workaround.

^{Reviewed by Cursor Bugbot for commit 1b76937. Configure here.}

Greptile Summary

This PR replaces model.setValue() with incremental model.applyEdits() in the Monaco editor's streaming sync effect to eliminate scroll/cursor flicker during Mothership file generation. The additive path correctly anchors against monacoValue (the live model snapshot) rather than lastSyncedContentRef.current, avoiding the stale-ref invariant from the previous round. The saveViewState/restoreViewState pair is correctly dropped since applyEdits preserves view state natively, and using applyEdits (vs. the earlier pushEditOperations) keeps streamed chunks out of the undo stack.

Confidence Score: 5/5

Safe to merge — targeted, well-reasoned fix with no new regressions introduced.

The change is small and focused. The two previously-raised P1 concerns (stale-ref invariant and undo-stack pollution) are both resolved in the final commit. The additive path correctly uses the live monacoValue snapshot, applyEdits bypasses the undo stack as intended, and removal of the save/restore view state pair is correct since applyEdits preserves scroll position natively. No P0 or P1 issues remain.

No files require special attention.

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[Effect fires: content changed or isStreamInteractionLocked changed] --> B{monacoValue === content?}
    B -- Yes --> C[Early return — no-op]
    B -- No --> D{isStreamInteractionLocked OR monacoValue === lastSyncedContentRef?}
    D -- No --> C2[Skip sync — user is actively editing]
    D -- Yes --> E[Capture scroll-stuck state]
    E --> F[suppressScrollListener = true]
    F --> G{content.startsWith monacoValue AND monacoValue shorter?}
    G -- Yes: additive --> H[applyEdits: insert suffix at end of last line]
    G -- No: non-additive --> I[applyEdits: replace full model range]
    H --> J[suppressScrollListener = false]
    I --> J
    J --> K[lastSyncedContentRef = content]

_{Reviews (3): Last reviewed commit: "fix(files): revert to applyEdits to avoi..." | Re-trigger Greptile}

@greptile

@cursor review

@greptile

@cursor review